-
-
Notifications
You must be signed in to change notification settings - Fork 364
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ExampleTestSuite: Also support mill
command in addition to ./mill
#2865
Conversation
mill
command in addition to ./mill
Should we support both? I feel like supporting 1 should be enough, and would be good to enforce some degree of consistency throughout the docsite which gets built from these test suites. If we want to use |
I wanted to go with |
Yeah, I think I got your initial motivation for I think keeping both variants supported is not really adding inconsistency. As we explain the user that we have that script in example |
sounds good to me, go for it |
Replace
./mill
withmill
in most examples, as this is easier to read. I kept the./mill
in the first simple scala project, which also describes that there is a./mill
script in each example archive.